Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix integration tests #18

Merged
merged 3 commits into from
Apr 22, 2024
Merged

Fix integration tests #18

merged 3 commits into from
Apr 22, 2024

Conversation

aneuwald-ctw
Copy link
Contributor

@aneuwald-ctw aneuwald-ctw commented Apr 22, 2024

User-Facing Changes
Integration tests were failing when running the command yarn run test:integration

Description
Update the integration tests to work with the MenuApp component, using custom data-testid parameter

@aneuwald-ctw aneuwald-ctw force-pushed the fix-integration-tests branch from 1c94ca1 to 9ce30b7 Compare April 22, 2024 16:29
Copy link
Contributor

@laisspportugal laisspportugal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@aneuwald-ctw aneuwald-ctw merged commit 6a7d668 into main Apr 22, 2024
7 of 10 checks passed
@aneuwald-ctw aneuwald-ctw deleted the fix-integration-tests branch April 22, 2024 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants